Skip to content

types: Improve the version specific SSOT docs #142

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 2, 2025

Conversation

tcharding
Copy link
Member

@tcharding tcharding commented Apr 30, 2025

The module docs in each version specific module (e.g. types::v17) are supposed to be the single source of truth for devs trying to work out which methods are supported and how.

Make an effort to better present the information.

Note that a bunch docs for methods marked as TODO may be incorrect re version vs version + model. This doesn't matter right now since they will be checked when implemented and also TODO implies no guarantees (in my mind anyway).

Note also issue #144. verify does not check the Returns column against the Method constructor type.

Close: #109

This TODO is described by issue rust-bitcoin#59 on github. Remove it.
The module docs in each version specific module (e.g. `types::v17`) are
supposed to be the single source of truth for devs trying to work out
which methods are supported and how.

Make an effort to better present the information.

Note that a bunch docs for methods marked as TODO may be incorrect re
version vs version + model. This doesn't matter right now since they
will be checked when implemented and also TODO implies no guarantees (in
my mind anyway).

Note also issue rust-bitcoin#144. `verify` does not check the Returns column against
the `Method` constructor type.

Close: rust-bitcoin#109
@tcharding tcharding force-pushed the 04-30-update-status-column branch from 2e315f5 to 4946066 Compare May 2, 2025 02:36
@tcharding tcharding changed the title types: Add Returns and Notes columns types: Improve the version specific SSOT docs May 2, 2025
@tcharding tcharding marked this pull request as ready for review May 2, 2025 02:37
@tcharding tcharding merged commit 3a059f8 into rust-bitcoin:master May 2, 2025
28 checks passed
@tcharding tcharding deleted the 04-30-update-status-column branch May 2, 2025 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

types: 'status' column is not that great
1 participant